Skip to content

migrate to Bazel build #2612

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 56 commits into
base: master
Choose a base branch
from
Draft

migrate to Bazel build #2612

wants to merge 56 commits into from

Conversation

@lyao-77
Copy link
Member

@lyao-77 lyao-77 commented Apr 17, 2023 •
edited
Loading

Prerequisite:

Running test:

  • bazel test //...

lyao-77 and others added 30 commits May 4, 2023 02:22
@lyao-77
first pass
0ab5193
@lyao-77
add BUILD files for each module
b02f235
@lyao-77
fixed build
9afc7bb
@lyao-77
some progress
ad41162
@lyao-77
revert avsc change
010af46
@lyao-77
add java test back
13178fb
@lyao-77
add bazelbersion for bazelisk
cbb3d68
@lyao-77
add proto build for client-encryption
085be41
@lyao-77
small fix on path
d92ee64
@lyao-77
fix some modules
47a7168
@lyao-77
some fix
a2f9a70
@lyao-77
small fix for client
39b7eea
@lyao-77
some fix
cfc3009
@lyao-77
some progress
a9de66c
@lyao-77
fix proto file deps
ec73673
@lyao-77
some more progress
bcf72e6
@lyao-77
Replace use of com_google_protobuf with rules_proto
4851422
@lyao-77
debug
7c2e2ca
@lyao-77
debug
9684754
@lyao-77
rename workspace
b91fbd7
@lyao-77
some fix
1b587c5
@lyao-77
use java_library from rules_jvm
eb485cc
@vinnybod @lyao-77
Get protobuf-serializer tests passing
4aaa4db
@lyao-77
linters config
6e6dc47
@vinnybod @lyao-77
fix the reference in RestApiSerializerTest
da8fdba
@lyao-77
fix proto import path with prefix and fix test failures
18b4a61
@lyao-77
fix test failures for schema-rules and protobuf-provider
b25fd06
@lyao-77
fix symlink path
ad65b56
@lyao-77
more test fix
9903229
@lyao-77
fix core
3e02911
vinnybod and others added 26 commits May 4, 2023 14:30
@vinnybod
initial swagger openapi spec generation
2e645d3
@vinnybod
get the file copy working
78bd2c4
@vinnybod
temp remove diff_test
49acac8
@vinnybod
some cleanup, support openapi.yaml
e267b62
@vinnybod
more code cleanup
2a05ed4
@vinnybod
fix tests and run buildifier
2e9acee
@vinnybod
Merge branch 'bazel_build' into openapi-gen
0f61a93
@vinnybod
remove json swagger
adf4bad
@vinnybod
move the openapi generation to the core module
feff0b9
@lyao-77
all tests passed including linters
3ef1c98
@lyao-77
new line and update spotbugs binary
268bff6
@vinnybod
remove local rule
b12d715
@vinnybod
fix spec
f84a11b
@vinnybod
Merge branch 'bazel_build' into openapi-gen
076ff7d
@lyao-77
add .bazel suffix back for workspace
b63e2e8
@vinnybod
Merge branch 'bazel_build' into openapi-gen
254b81b
@vinnybod
use swagger_core_version
b4fb127
@lyao-77
Temp commit
0874017
@vinnybod
update confluent rules to 0.0.5
3392b9e
@lyao-77
use "assemble_package" macro
8f69652
@vinnybod
support minimal maven usage to build the kafka-schema-registry-maven-…
f39b344 
…plugin
@lyao-77
merge
65bd57c
@lyao-77
use glob and fix some issues
563eb2b
@lyao-77
Revert "use glob and fix some issues"
c4396b2 
This reverts commit 563eb2b.
@lyao-77
use glob
edae8ec
@lyao-77
resolve conflict
cf993c0
@cla-assistant
Copy link

cla-assistant bot commented Sep 25, 2023 •
edited
Loading

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ vinnybod
❌ lyao-77
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lyao-77 @vinnybod